-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
enhance(oidc): add more claims #1172
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1172 +/- ##
==========================================
- Coverage 52.85% 52.84% -0.02%
==========================================
Files 551 551
Lines 19182 19199 +17
==========================================
+ Hits 10139 10146 +7
- Misses 8479 8489 +10
Partials 564 564
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
one minor thing, otherwise lgtm
Is there an intention to follow up this PR with the API implementation here? |
Whoops, got confused and thought |
closes go-vela/community#996
Note: I intended to include whether the build was triggered from a fork, but it seems we don’t currently store that information in the database. Therefore, I decided to omit that detail for now. If someone wants to add this feature in the future, they will also need to update the database to capture this data.